-
Notifications
You must be signed in to change notification settings - Fork 5.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ACSS Swagger Changes in PUT SVI API and child resources #21604
ACSS Swagger Changes in PUT SVI API and child resources #21604
Conversation
Hi, @dindurkhya Thanks for your PR. I am workflow bot for review process. Here are some small tips. Any feedback about review process or workflow bot, pls contact swagger and tools team. [email protected] |
Swagger Validation Report
|
compared swaggers (via Oad v0.10.2)] | new version | base version |
---|---|---|
SAPVirtualInstance.json | 2021-12-01-preview(a12a6d3) | 2021-12-01-preview(main) |
️️✔️
Breaking Change(Cross-Version) succeeded [Detail] [Expand]
There are no breaking changes.
️️✔️
CredScan succeeded [Detail] [Expand]
There is no credential detected.
️⚠️
LintDiff: 0 Warnings warning [Detail]
compared tags (via openapi-validator v2.0.0) | new version | base version |
---|---|---|
package-2021-12-01-preview | package-2021-12-01-preview(a12a6d3) | package-2021-12-01-preview(main) |
The following errors/warnings exist before current PR submission:
Rule | Message |
---|---|
Consider using x-ms-client-flatten to provide a better end user experience Location: Microsoft.Workloads/preview/2021-12-01-preview/SAPVirtualInstance.json#L1797 |
|
Booleans properties are not descriptive in all cases and can make them to use, evaluate whether is makes sense to keep the property as boolean or turn it into an enum. Location: Microsoft.Workloads/preview/2021-12-01-preview/SAPVirtualInstance.json#L1935 |
|
Booleans properties are not descriptive in all cases and can make them to use, evaluate whether is makes sense to keep the property as boolean or turn it into an enum. Location: Microsoft.Workloads/preview/2021-12-01-preview/SAPVirtualInstance.json#L2157 |
|
Booleans properties are not descriptive in all cases and can make them to use, evaluate whether is makes sense to keep the property as boolean or turn it into an enum. Location: Microsoft.Workloads/preview/2021-12-01-preview/SAPVirtualInstance.json#L2161 |
|
Booleans properties are not descriptive in all cases and can make them to use, evaluate whether is makes sense to keep the property as boolean or turn it into an enum. Location: Microsoft.Workloads/preview/2021-12-01-preview/SAPVirtualInstance.json#L2799 |
|
Booleans properties are not descriptive in all cases and can make them to use, evaluate whether is makes sense to keep the property as boolean or turn it into an enum. Location: Microsoft.Workloads/preview/2021-12-01-preview/SAPVirtualInstance.json#L3550 |
️️✔️
Avocado succeeded [Detail] [Expand]
Validation passes for Avocado.
️️✔️
ApiReadinessCheck succeeded [Detail] [Expand]
️⚠️
~[Staging] ServiceAPIReadinessTest: 0 Warnings warning [Detail]
API Test is not triggered due to precheck failure. Check pipeline log for details.
️️✔️
~[Staging] SwaggerAPIView succeeded [Detail] [Expand]
️️✔️
ModelValidation succeeded [Detail] [Expand]
Validation passes for ModelValidation.
️️✔️
SemanticValidation succeeded [Detail] [Expand]
Validation passes for SemanticValidation.
️️✔️
PoliCheck succeeded [Detail] [Expand]
Validation passed for PoliCheck.
️️✔️
PrettierCheck succeeded [Detail] [Expand]
Validation passes for PrettierCheck.
️️✔️
SpellCheck succeeded [Detail] [Expand]
Validation passes for SpellCheck.
️️✔️
Lint(RPaaS) succeeded [Detail] [Expand]
Validation passes for Lint(RPaaS).
️️✔️
CadlValidation succeeded [Detail] [Expand]
Validation passes for CadlValidation.
️️✔️
PR Summary succeeded [Detail] [Expand]
Validation passes for Summary.
Swagger Generation Artifacts
|
Generated ApiView
|
NewApiVersionRequired reason: |
…)" This reverts commit 58f076e.
…e#21604)" (Azure#21662) This reverts commit 58f076e.
…es (Azure#21604)" (Azure#21662)" (Azure#21663) This reverts commit 495363b.
…e#21604)" (Azure#21662) This reverts commit 58f076e.
…es (Azure#21604)" (Azure#21662)" (Azure#21663) This reverts commit 495363b.
SAP Disk Customization
Add an optional property "Disk Configuration" in the PUT SVI payload. It will not break any existing customer flow as for existing customers this value is set to a default recommended configuration. For New VIS PUT calls, the user can choose to customize and provided changes.
Adding new property DiskConfigurationDetails in PUT SVI API input pay… by dindurkhya · Pull Request #9563 · Azure/azure-rest-api-specs-pr (github.com)
SAP Transport Fileshare Management
Add an optional property "StorageConfiguration" in InfrastructureConfiguration. The default behavior for when this property is not present is defined and it will continue to work for existing API/CLI clients if they do not make any changes to their payloads. It just adds new functionality for users who want to leverage it.
Swagger Changes for SAP Trans to 2021 Version by shreyks · Pull Request #9784 · Azure/azure-rest-api-specs-pr (github.com)
SAP LoadBalancer and Storage Details
Add an optional, read only property "LoadBalancerDetails" in SapCentralServerProperties, SapDatabaseProperties.
Add an Optional, read only property "StorageDetails" in CentralServerVmDetails, DatabaseVmDetails, SapApplicationServerProperties.
For Existing Sap Virtual Instances either these properties won't be present or will populated for relevant HA, Distributed type VISs.
Bmopati/LB and storage details changes for 2022-11-01 pr by bmopati · Pull Request #9464 · Azure/azure-rest-api-specs-pr (github.com)
Breaking Change Justification
The default behavior for when this property is not present is defined and it will continue to work for existing API/CLI clients if they do not make any changes to their payloads. It just adds new functionality for users who want to leverage it.
Switch to "Preview" on this description then select one of the choices below.
Click here to open a PR for a Data Plane API.
Click here to open a PR for a Control Plane (ARM) API.